Skip to content

Update ingress-traefik.yaml#191

Open
ar0k wants to merge 1 commit intokeephq:mainfrom
ar0k:patch-1
Open

Update ingress-traefik.yaml#191
ar0k wants to merge 1 commit intokeephq:mainfrom
ar0k:patch-1

Conversation

@ar0k
Copy link
Copy Markdown

@ar0k ar0k commented Mar 3, 2026

change ingressClassName value for helm variable value.
this is mandatory if ingressclassname is not traefik

Closes #

📑 Description

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

ℹ Additional Information

change ingressClassName value

Signed-off-by: ar0k <mguerchet@yahoo.fr>
@ar0k ar0k requested a review from talboren as a code owner March 3, 2026 05:06
@shahargl
Copy link
Copy Markdown
Member

shahargl commented Mar 3, 2026

but you are changing charts/keep/templates/ingress-traefik.yaml so it only applies when its traefik - what do I miss

@ar0k
Copy link
Copy Markdown
Author

ar0k commented Mar 4, 2026

"I set the ingressClassName value using the variable from the values file instead of hardcoding 'traefik'. In my Kubernetes setup, the ingress class is not named 'traefik'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants